Open
Conversation
1hyok
reviewed
Mar 31, 2026
| .padding(24.dp) | ||
| .fillMaxSize() | ||
| ) { | ||
| Row( |
There was a problem hiding this comment.
탑바 부분과 스크롤되는 콘텐트 부분을 가독성과 유지 보수를 위해 컴포저블 함수로 분리하는 것이 좋습니다!
1hyok
reviewed
Mar 31, 2026
| modifier = Modifier | ||
| .windowInsetsPadding(WindowInsets.safeDrawing) | ||
| .padding(24.dp) | ||
| .fillMaxSize() |
There was a problem hiding this comment.
패딩, 필맥스 사이즈의 순서가 어떻든 이 상황에서는 결과가 같습니다. 하지만 논리적인 흐름이 자연스럽게 읽히도록 필맥스사이즈를 먼저 적용하는 것이 일반적입니다. 코틀린이 선언형 프로그래밍인 만큼 꽉 찬 화면에 패딩을 적용하겠다라고 하는 의도대로 순서를 적용하는 것이 좋습니다!
1hyok
reviewed
Mar 31, 2026
| ) { | ||
| items(articles) { article -> | ||
| Row( | ||
| modifier = Modifier.height(96.dp) |
There was a problem hiding this comment.
하드 코딩을 피하기 위한 방법 중 하나로 IntrinsicSize.Min를 쓰는 것이 맞습니다. 하지만 현재 피그마 디자인 상 간격이 다 다르기 때문에 스페이스 비트윈 대신 스페이서를 적용하면 하드 코딩 문제가 생기지 않습니다!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1. 미션
2. 구현에 대한 설명
1. 연락처
데이터 모델링
Call데이터 클래스를 정의했습니다.callList객체로 생성하여 리스트 형태로 관리했습니다.LazyColumn을 통한 성능 최적화
LazyColumn을 사용했습니다.items(callList)를 활용해 반복되는 UI 컴포넌트를 동적으로 생성했습니다.구분선
HorizontalDivider(1.dp)를 배치하여 항목 간 경계를 나눴습니다.2. 뉴스
데이터 모델링
News클래스와 이를 포함하는 기사 정보Article클래스를 분리하여 설계했습니다.articles와categories리스트를 생성하여 기사들과 기사의 카테고리를 표시하도록 했습니다.LazyRow, LazyColumn을 통한 성능 최적화
LazyRow(카테고리) : "All", "Sports" 등의 카테고리를 가로 스크롤로 배치하여 다양한 카테고리를 제공합니다.LazyColumn(뉴스 기사) : 세로 스크롤을 통해 많은 양의 뉴스 기사 리스트를 성능 저하 없이 렌더링하도록 했습니다.UI 효과 및 배치
Shadow를 사용하여 그림자 효과를 내었습니다.Row안에Column을 중첩하여 이미지와 기사 본문이 나란히 배치되는 뉴스 피드를 완성했습니다.3. 스크린샷 & 실행영상
4. 기타